Skip to content

Populate l1_transaction.amount for executed withdrawals#30

Open
ztepler wants to merge 1 commit intomasterfrom
fix-withdrawal-l1-amount
Open

Populate l1_transaction.amount for executed withdrawals#30
ztepler wants to merge 1 commit intomasterfrom
fix-withdrawal-l1-amount

Conversation

@ztepler
Copy link
Copy Markdown
Member

@ztepler ztepler commented Apr 21, 2026

No description provided.

TezosWithdrawOperation records created in on_rollup_execute were always
persisted with amount=NULL, so bridge_operation.withdrawal.l1_transaction.amount
was missing via GraphQL for classic XTZ and FA withdrawals as well as for
fast-withdrawal service-provider reimbursements. Extract the L1-native amount
from the outbox message's ticket payload and store it on the L1 record.

Also cache the two outbox Michelson types at module level to avoid
recompiling them on every parse; reused in OutboxParametersHash as well.

Assisted-By: claude-opus-4-7[1m]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant